Skip to content

Feat: FetchGraphToDepth() to fetch a graph to a given depth#5134

Closed
hsanjuan wants to merge 1 commit intomasterfrom
feat/enumerate-to-depth
Closed

Feat: FetchGraphToDepth() to fetch a graph to a given depth#5134
hsanjuan wants to merge 1 commit intomasterfrom
feat/enumerate-to-depth

Conversation

@hsanjuan
Copy link
Contributor

@hsanjuan hsanjuan commented Jun 18, 2018

This comes in the context of #5133. It enables Merkledag to fetch
DAGs down to a given depth.

Note that actual usage of depth is expected to be 1, or 2 (and not
an arbitrarily high value), thus I have opted to not complicate things with
branch-pruning optimizations. They can be introduced if they are ever needed
at another point in time.
actually there is branch pruning in the new PR.

@hsanjuan hsanjuan self-assigned this Jun 18, 2018
@hsanjuan hsanjuan requested a review from Kubuxu as a code owner June 18, 2018 14:08
@ghost ghost added the status/in-progress In progress label Jun 18, 2018
@hsanjuan hsanjuan force-pushed the feat/enumerate-to-depth branch 2 times, most recently from d9be5a0 to 6dc6748 Compare June 18, 2018 14:11
This comes in the context of #5133. It enables Merkledag to fetch
DAGs down to a given depth.

Note that actual usage of depth is expected to be 1, or 2 (and not
an arbitrarily high value), thus I have opted to not complicate things with
branch-pruning optimizations. They can be introduced if they are ever needed
at another point in time.

License: MIT
Signed-off-by: Hector Sanjuan <code@hector.link>
@hsanjuan hsanjuan force-pushed the feat/enumerate-to-depth branch from 6dc6748 to 8e34775 Compare June 18, 2018 14:16
@hsanjuan
Copy link
Contributor Author

Superseeded by #5142

@hsanjuan hsanjuan closed this Jun 20, 2018
@ghost ghost removed the status/in-progress In progress label Jun 20, 2018
@hsanjuan hsanjuan deleted the feat/enumerate-to-depth branch June 20, 2018 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant